fix python packaging using latest methods#79
Conversation
|
|
|
If I follow the latest python packaging guide (Link in Commit), this is the minimal change that was required. I just moved the sources to |
|
Got it, thanks for the explanation. |
The changes were required, as I mentioned in commit, to package it for debian and others. |
|
I would suggest do not merge it yet. because the setup.py needs to be cleaned. It would be highly appreciated if you could test the branch. just use |
|
I would also suggest while merging use the Squash and merge option rather than creating a new commit |
|
@brlin-tw you can merge the branch. I have completed all the necessary changes and it works fine. |
|
@brlin-tw A gente reminder that the PR is ready for merge. Thank you. |
|
@brlin-tw Hi. Is this PR going to get merged or should I close this as incomplete? |
|
Hi @hsbasu, thanks for PR. |
- Change python packaging method following instructions from https://packaging.python.org/en/latest/tutorials/packaging-projects - Remove scripts as they can be created automatically from setup.cfg - move the sources to src - setup.py: remove captialization in package names, conflicts with other packaging methods like debian's - modify licence to gpl3+ - Add setup.cfg and pyproject.toml - modify MANIFEST.in to include assets - WoeUSB-ng.desktop: add version and change icon; fixes WoeUSB#11
2005857 to
f9d54c9
Compare
|
FYI, I use this PR without issue to build the aur package available at https://aur.archlinux.org/packages/woeusb-ng. The pkgbuild file that show the build process is available here. I'll subscribe to this thread to be notified about the merge to update the package. |
|
@brlin-tw this PR would greatly help every package maintainer. the one with Arch-Linux reported it. I can confirm this for NixOS. It would be very nice to merge this and reduce the load for package-maintainers. Otherwise they might consider this project as archived. |
https://packaging.python.org/en/latest/tutorials/packaging-projects/
packaging methods like debian's